Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix buried larva spawn grace period at start of round #4988

Merged
merged 2 commits into from
Nov 22, 2023

Conversation

Drulikar
Copy link
Contributor

@Drulikar Drulikar commented Nov 21, 2023

About the pull request

Looking at the code again, #4502 is unintentionally eliminating the buried larva grace period when there is no core. The first 30 minutes should have a core be optional (because the hive is still being established), but the queue requires a core (or nested bodies) to dequeue players from the queue. So as soon as there is one player in queue, buried spawns during the no-core grace period would be stopped.

This PR makes it so the queue will only prevent manual buried spawns when there is a core.

Explain why it's good for the game

Fixes a situation that needlessly hinders xeno start if the hive puts off creating a core.

Also the logic was getting fairly complex here so the early returns should make it more manageable.

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑 Drathek
fix: Fixed buried larva spawn grace period at start of round if there is a queue: Now join as xeno when there's a queue will only prevent buried larva spawns if there is no core.
/:cl:

…irst 30 minutes of the round

Cleanup logic to use early returns instead
@github-actions github-actions bot added the Fix Fix one bug, make ten more label Nov 21, 2023
@harryob harryob added this pull request to the merge queue Nov 22, 2023
Merged via the queue into cmss13-devs:master with commit fa3745b Nov 22, 2023
28 checks passed
cm13-github added a commit that referenced this pull request Nov 22, 2023
@Drulikar Drulikar deleted the Fix_Early_Game_Larva_Edgecase branch November 22, 2023 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Fix one bug, make ten more
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants